Skip to content

Conversation

ianayl
Copy link
Contributor

@ianayl ianayl commented Mar 19, 2025

This PR:

This PR serves as a stepping stone for combining the SYCL benchmarking CI with the UR benchmarking CI: Given that #17229 is massive, I am splitting that PR into smaller individual PRs so that reviewers have an easier time.

pbalcer added 6 commits March 19, 2025 15:47
This patch improves numerous aspects on how the benchmarking
results are visualized:
 - rewrites the way HTML charts are generated, using a library (Chart.js)
 that's both easier to use and more visually pleasing.
 The new HTML page also now decouples data from the HTML itself,
 leading to faster load times and the ability to fetch data
 from remote sources.
 - The markdown output now contains a failures section that
 lists all benchmarks that failed for a given run. This will be
 a helpful for developers during PR testing.
 - Benchmarks can now have description that's displayed on the page.
 - And many more minor improvements.
@ianayl ianayl temporarily deployed to WindowsCILock March 19, 2025 23:14 — with GitHub Actions Inactive
@ianayl ianayl closed this Mar 19, 2025
@ianayl ianayl deleted the integrate-ur-benchmarking branch March 19, 2025 23:26
@ianayl ianayl temporarily deployed to WindowsCILock March 19, 2025 23:33 — with GitHub Actions Inactive
@ianayl ianayl temporarily deployed to WindowsCILock March 19, 2025 23:33 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants